Skip to content

Conversation

@lwjohnst86
Copy link
Member

Description

Very small fix to use properties in read_json().

Needs a quick review.

Checklist

  • Ran just run-all

@lwjohnst86 lwjohnst86 requested a review from a team as a code owner October 28, 2025 09:00
@lwjohnst86 lwjohnst86 changed the base branch from feat/implement-error-true-argument to main October 28, 2025 09:03
@lwjohnst86 lwjohnst86 marked this pull request as draft October 28, 2025 09:03
@lwjohnst86 lwjohnst86 moved this from Todo to In Progress in Iteration planning Oct 28, 2025
@lwjohnst86
Copy link
Member Author

I messed up, this has commits from another PR. Will wait until that one is merged before converting this one to be ready for review.

@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Nov 4, 2025
@lwjohnst86 lwjohnst86 marked this pull request as ready for review November 4, 2025 10:35
@martonvago
Copy link
Contributor

Sorry, are you waiting on me for this?

@lwjohnst86
Copy link
Member Author

@martonvago yes! Or someone 😛

Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!!

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Nov 7, 2025
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Nov 14, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More like a question 😅

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Nov 14, 2025
lwjohnst86 and others added 3 commits November 14, 2025 12:22
Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
@lwjohnst86 lwjohnst86 moved this from In Progress to In Review in Iteration planning Nov 14, 2025
Copy link
Contributor

@martonvago martonvago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Iteration planning Nov 14, 2025
@lwjohnst86 lwjohnst86 merged commit d75d075 into main Nov 14, 2025
6 checks passed
@lwjohnst86 lwjohnst86 deleted the refactor/rename-to-properties-in-read-json branch November 14, 2025 15:31
@github-project-automation github-project-automation bot moved this from In Progress to Done in Iteration planning Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants